Skip to content

Async streaming extension methods + ConfigureAwaitFalse improvements#77

Open
FluttercordKN wants to merge 1 commit intoKnoema:masterfrom
FluttercordKN:master
Open

Async streaming extension methods + ConfigureAwaitFalse improvements#77
FluttercordKN wants to merge 1 commit intoKnoema:masterfrom
FluttercordKN:master

Conversation

@FluttercordKN
Copy link
Copy Markdown
Contributor

No description provided.

# Visual Studio 15
VisualStudioVersion = 15.0.28010.2046
# Visual Studio Version 16
VisualStudioVersion = 16.0.31112.23
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This probably stops the project opening on old VS versions


var result = await instance.GetDataAsync(request).ToListAsync();

Console.WriteLine(result.Count);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no assert. Maybe it's worth adding specific elements and check the expected ts count. No exceptions are good, but it might be not enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants